Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Streamline nox sessions #32

Merged
merged 2 commits into from
Dec 12, 2023
Merged

ci: Streamline nox sessions #32

merged 2 commits into from
Dec 12, 2023

Conversation

ESKYoung
Copy link
Owner

Summary

nox sessions are currently bloated, with some unnecessary installs. Coupled with a need to better handle Poetry groups, the sessions have been refactored, and streamlined.

Checklists

I/we (contributor) confirm that the code, and analyses in this Pull Request (developments) meets the following requirements:

  • code runs
  • developments are ethical, and secure
  • contributor has made proportionate checks that the developments are correct
  • minimum usable documentation is written in plain English in the docs folder
  • all pre-commit hooks pass
  • test suite passes
  • code coverage is maintained, or increased

Additional comments

N/A

Remove `nox-poetry` requirement, and reduce the number of steps in
GitHub Actions.
@ESKYoung ESKYoung added the ci Changes to CI configuration files and scripts label Dec 12, 2023
@ESKYoung ESKYoung self-assigned this Dec 12, 2023
@ESKYoung ESKYoung merged commit 3012b7a into main Dec 12, 2023
10 checks passed
@ESKYoung ESKYoung deleted the streamline-nox branch December 12, 2023 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Changes to CI configuration files and scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant